Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Multishot POC #36

Closed
wants to merge 17 commits into from
Closed

Multishot POC #36

wants to merge 17 commits into from

Conversation

kyay10
Copy link

@kyay10 kyay10 commented Oct 1, 2024

Fixes #34

I have no expectation that this PR will be merged at all. This likely needs more design work, hence it's just a POC. I've made sure that there's near feature-compatibility, but of course the API had to change somewhat. The current version of kontinuity doesn't support Native because I haven't figured out how to shallow clone Continuations yet (I haven't tried hard enough though, so suggestions are very welcome).
I had to modify some of the tests that e.g. expected failures for things that are supported now. Also, tests for parameter being used inside of a Sequence couldn't work anymore because of the usage of suspend, and hence they had to be removed.

Pass HandlerPrompt through to ParameterizeState
Remove ParameterState.isDeclared and corresponding tests
Remove ParameterState.getArgument in favour of the property argument
Delete irrelevant tests
Ensure that only one single `runTest` is triggered per test, and that its result is returned
Restore hasBeenUsed status on each iteration
Make DecoratorScope @RestrictsSuspension
Remove unused ParameterState iteration functionality
Add ParameterizeDecorator
Fix testing for empty iterations
Fix iteration count miscounting and add a test for it
@kyay10
Copy link
Author

kyay10 commented Oct 1, 2024

Closed in favour of #37 because I hadn't rebased correctly (oops)

@kyay10 kyay10 closed this Oct 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implement parameterize as multi-shot coroutine
1 participant